Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send Welcome Kits by Household #30

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

bencap
Copy link
Collaborator

@bencap bencap commented Sep 28, 2022

As the Cascadia study was getting off the ground, we sent welcome kits to a participant if
they were fully consented and enrolled even if the household wasn't. Now that the study is
further along, this creates additional shipments that we can eliminate. This change only
makes a participant eligible for a welcome shipment once their whole household is fully
consented and enrolled, even if that participant was consented and enrolled before that.

This change also includes a minor change to resupply logic, only making a participant eligible once they have been assigned barcodes.

Note that we don't need to merge this until we get approval from Zack. This change to study logic is planned but not yet desired.

A previous version of the resupply function would resupply participants even if they
had zero previous swabs assigned to them simply because this zero swabs was below
the threshold value. This change makes it explicit that participants should have
some number of barcodes before being eligible for a resupply shipment.
As the Cascadia study was getting off the ground, we sent welcome kits to a participant if
they were fully consented and enrolled even if the household wasn't. Now that the study is
further along, this creates additional shipments that we can eliminate. This change only
makes a participant eligible for a welcome shipment once their whole household is fully
consented and enrolled, even if that participant was consented and enrolled before that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant